-
-
Notifications
You must be signed in to change notification settings - Fork 674
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ability to indent/un-indent multiple list items #1642
Add ability to indent/un-indent multiple list items #1642
Conversation
🦋 Changeset detectedLatest commit: 94cd1a4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 8 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
eaeba10
to
4b43c4b
Compare
@zbeyens i'm still working on some more tests and if they pass, will make it ready-for review I am seeing that if you have a list like this:
(with focus/anchor as shown by |), and pressing tab, the editor itself loses focus (or in playgrounds, it doesn't do anything). Surprisingly, the following works fine'ish:
This results in the incorrect:
I was expecting it to not change (since we can't double indent on the list)
Here are some other examples: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, great work!
@zakishaheen just needs a changeset as mentioned by the comment above |
Changeset added @zbeyens |
Description
This fixes a whole class of list indentation bugs that are introduced due to "hanging" nodes.
There is still a bug in Slate's hang code for which @davisg123 created this PR.
See changesets.
Before:
CleanShot.2022-06-30.at.17.41.03.mp4
After:
CleanShot.2022-06-30.at.17.39.58.mp4